I can not remove only one place，follow :
path: testutil/cli/cmt_mocks.go
func: func (m MockCometRPC) ABCIQueryWithOptions

because I need to modify package of cometbft

super nice docs, it looks good to me. It doesnt seem like we use the json encoding anywhere i can see.

It seems that cmtbytes does not use func of MarshalJSON() nor UnmarshalJSON(), only use of the func String(), so that I do not use json encoding.

fmt.Sprintf("%X", e.Hash()) seems to be faster than strings.ToUpper(hex.EncodeToString(e.Hash())). Any objections to using fmt?

Note: hex.EncodeToString by itself is faster but because we do strings.ToUpper it's the other way around.

strings.ToUpper(hex.EncodeToString(e.Hash()))

@facundomedica I have made a bench test,
`func Benchmark_Fmt(b *testing.B) {
e := []byte("hello")
for i := 0; i < b.N; i++ {
fmt.Sprintf("%X", e)
}
}

func Benchmark_Hex(b *testing.B) {
e := []byte("hello")
for i := 0; i < b.N; i++ {
hex.EncodeToString(e)
}
}

func Benchmark_Shex(b *testing.B) {
e := []byte("hello")
for i := 0; i < b.N; i++ {
strings.ToUpper(hex.EncodeToString(e))
}
}`
and the result as follow:

goos: darwin
goarch: amd64
pkg: mytest/struct_01
cpu: Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
Benchmark_Fmt
Benchmark_Fmt-16 9034696 125.7 ns/op
Benchmark_Hex
Benchmark_Hex-16 49447454 23.87 ns/op
Benchmark_Shex
Benchmark_Shex-16 18752376 64.16 ns/op
PASS

It seems that strings.ToUpper(hex.EncodeToString(e.Hash())) is faster than fmt.Sprintf("%X", e.Hash())`

I am going to merge this now (we can improve in a follow-up).
Thank you, @chixiaowen, for your contribution!